Skip to content

Conversation

@Eucha09
Copy link
Collaborator

@Eucha09 Eucha09 commented Aug 7, 2023

No description provided.

@Eucha09 Eucha09 requested a review from gbe0808 August 7, 2023 03:42
@Eucha09 Eucha09 marked this pull request as draft August 7, 2023 04:58
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ch09에서 수정하면 충돌날 수 있으니까 빼는 게 좋을듯

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ch09에서 수정하면 충돌날 수 있으니까 이것까지 빼는 게 좋을듯

Comment on lines +7 to +8
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
for i in range(len(word)-1, -1, -1):
alp_value[ord(word[i]) - ord('A')] += place_value
for x in word[::-1]:
alp_value[ord(x) - ord('A')] += place_value

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우리 struct 이름 첫 글자는 대문자로 하기로 했었나?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

나는 뭔가 for voted_number in voted_number_list가 더 나아보이는듯?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😲

Comment on lines +1 to +5
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이런건 괜찮나?

Suggested change
def how_many_pass(given_time, time_per_pass):
count = 0
for t in time_per_pass:
count += given_time // t
return count
def how_many_pass(given_time, time_per_pass):
return sum(given_time // t for t in time_per_pass)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants